Skip to content

feat: switch link direction between feat and log_int_arc - #675

Merged
AlexanderLanin merged 3 commits into
eclipse-score:mainfrom
etas-contrib:switch_feat_log_int_arc_link_direction
Jul 30, 2026
Merged

feat: switch link direction between feat and log_int_arc#675
AlexanderLanin merged 3 commits into
eclipse-score:mainfrom
etas-contrib:switch_feat_log_int_arc_link_direction

Conversation

@RolandJentschETAS

@RolandJentschETAS RolandJentschETAS commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

📌 Description

This pull request updates the requirements and metamodel to support a migration in the directionality of the link between feat and logic_arc_int. The changes ensure that both the old and new link directions are supported as optional during the transition, and update documentation and tests accordingly.

Requirements and Metamodel Migration:

  • Updated the requirements CSV table in requirements.rst to add logic_arc_int included_by feat as an optional link and clarified that both directions are accepted during migration. Also added a note explaining the link direction switch to help with traceability and migration. [1] [2]
  • Changed metamodel.yaml so that the old includes link is now optional, and the new included_by link is also optional for logic_arc_int during migration. Added comments to clarify the migration status of both directions. [1] [2]

Test Updates:

  • Updated architecture tests in architecture_tests.rst to include the new :included_by: field for relevant logic_arc_int entries, ensuring tests cover the new migration path. [1] [2]

🚨 Impact Analysis

  • This change does not violate any tool requirements and is covered by existing tool requirements
  • This change does not violate any design decisions
  • Otherwise I have created a ticket for new tool qualification

✅ Checklist

  • Added/updated documentation for new or changed features
  • Added/updated tests to cover the changes
  • Followed project coding standards and guidelines

@github-actions

github-actions Bot commented Jul 30, 2026 •
edited
Loading

Copy link
Copy Markdown

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //src:license-check

Status: ⚠️ Needs Review

Click to expand output 
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: 54cc979d-891e-4217-bb08-32c50673a609
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
WARNING: Target pattern parsing failed.
ERROR: Skipping '//src:license-check': no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
ERROR: no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
INFO: Elapsed time: 6.480s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

AlexanderLanin
AlexanderLanin reviewed Jul 30, 2026
View reviewed changes
Comment thread docs/internals/requirements/requirements.rst Outdated
RolandJentschETAS and others added 2 commits July 30, 2026 15:30
@RolandJentschETAS @AlexanderLanin
Update docs/internals/requirements/requirements.rst
300e756 
Co-authored-by: Alexander Lanin <Alexander.Lanin@etas.com>
Signed-off-by: RolandJentschETAS <135332348+RolandJentschETAS@users.noreply.github.com>
@RolandJentschETAS
fix build
894442f
@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

@AlexanderLanin AlexanderLanin changed the title switch link direction between feat and log_int_arc feat: switch link direction between feat and log_int_arc Jul 30, 2026
@AlexanderLanin
AlexanderLanin merged commit 4a8197f into eclipse-score:main Jul 30, 2026
12 of 14 checks passed
@AlexanderLanin
AlexanderLanin deleted the switch_feat_log_int_arc_link_direction branch July 30, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AlexanderLanin AlexanderLanin AlexanderLanin approved these changes
@MaximilianSoerenPollak MaximilianSoerenPollak Awaiting requested review from MaximilianSoerenPollak MaximilianSoerenPollak is a code owner
@dcalavrezo-qorix dcalavrezo-qorix Awaiting requested review from dcalavrezo-qorix dcalavrezo-qorix is a code owner
@nradakovic nradakovic Awaiting requested review from nradakovic nradakovic is a code owner

Assignees

No one assigned

Labels

None yet

Projects

INF - Infrastructure Community
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@RolandJentschETAS @AlexanderLanin